Skip to content

Update Azure.Identity to 1.16.0 & Azure.Core to 1.49.0 #11404

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 5 commits into
base: dev
Choose a base branch
from
Open

Update Azure.Identity to 1.16.0 & Azure.Core to 1.49.0 #11404

wants to merge 5 commits into from

Conversation

cataggar
Copy link
Member

@cataggar cataggar commented Oct 2, 2025

Issue describing the changes in this PR

resolves #11402

Pull request checklist

IMPORTANT: Currently, changes must be backported to the in-proc branch to be included in Core Tools and non-Flex deployments.

  • Backporting to the in-proc branch is not required
    • Otherwise: Link to backporting PR
  • My changes do not require documentation changes
    • Otherwise: Documentation issue linked to PR
  • My changes should not be added to the release notes for the next release
    • Otherwise: I've added my notes to release_notes.md
  • My changes do not need to be backported to a previous version
    • Otherwise: Backport tracked by issue/PR #issue_or_pr
  • My changes do not require diagnostic events changes
    • Otherwise: I have added/updated all related diagnostic events and their documentation (Documentation issue linked to PR)
  • I have added all required tests (Unit tests, E2E tests)

Additional information

Additional PR information

Copilot AI and others added 2 commits October 2, 2025 13:04
@Copilot @cataggar
Update Azure.Identity to 1.16.0 to support AZURE_TOKEN_CREDENTIALS en…
ba930cf 
…vironment variable (#2)

* Initial plan

* Update Azure.Identity from 1.14.2 to 1.16.0

Co-authored-by: cataggar <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: cataggar <[email protected]>
@Copilot @cataggar
Update release_notes.md for Azure.Identity 1.16.0 package update
3294da2 
* Initial plan

* Add release note for Azure.Identity 1.16.0 update

Co-authored-by: cataggar <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: cataggar <[email protected]>
@cataggar cataggar requested a review from a team as a code owner October 2, 2025 20:10
@Copilot Copilot AI review requested due to automatic review settings October 2, 2025 20:10
Copilot
Copilot AI reviewed Oct 2, 2025
View reviewed changes
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Updates the Azure.Identity package from version 1.14.2 to 1.16.0 to support the AZURE_TOKEN_CREDENTIALS environment variable functionality.

  • Package version update from 1.14.2 to 1.16.0 in project files
  • Added release notes entry documenting the change

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/WebJobs.Script/WebJobs.Script.csproj Updated Azure.Identity package reference to version 1.16.0
src/WebJobs.Script.WebHost/WebJobs.Script.WebHost.csproj Updated Azure.Identity package reference to version 1.16.0
release_notes.md Added release note documenting the Azure.Identity update

@cataggar cataggar changed the title Azure.identity Update Azure.Identity to 1.16.0 Oct 2, 2025
@jviau jviau requested a review from mattchenderson October 2, 2025 21:35
@jviau
Copy link
Contributor

jviau commented Oct 2, 2025

/azp run host.public

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jviau
Copy link
Contributor

jviau commented Oct 2, 2025

@cataggar the build failed due to a package downgrade error. You should be able to get the same error with a local dotnet build.

@cataggar cataggar changed the title Update Azure.Identity to 1.16.0 Update Azure.Identity to 1.16.0 & Azure.Core to 1.49.0 Oct 3, 2025
@cataggar
Copy link
Member Author

cataggar commented Oct 3, 2025

/azp run host.public

@azure-pipelines Azure Pipelines
Copy link

Commenter does not have sufficient privileges for PR 11404 in repo Azure/azure-functions-host

@cataggar
Copy link
Member Author

cataggar commented Oct 3, 2025

@jviau It restores and builds locally now. I don't have permissions to start CI. Can you kick off another run or can I get permissions?

@jviau
Copy link
Contributor

jviau commented Oct 3, 2025

/azp run host.public

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jviau
Copy link
Contributor

jviau commented Oct 3, 2025

@cataggar another change will be needed. We have a unit test which verifies our transitive dependency closure to identify changes like this. We need to update a file to signal to the test this change is intentional. Please replace the contents of test/WebJobs.Script.Tests/Microsoft.Azure.WebJobs.Script.WebHost.deps.json with this artifact

@jviau
Copy link
Contributor

jviau commented Oct 3, 2025

/azp run host.public

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@cataggar
Copy link
Member Author

cataggar commented Oct 3, 2025

@cataggar another change will be needed. We have a unit test which verifies our transitive dependency closure to identify changes like this. We need to update a file to signal to the test this change is intentional. Please replace the contents of test/WebJobs.Script.Tests/Microsoft.Azure.WebJobs.Script.WebHost.deps.json with this artifact

Thanks @jviau . This is done.

@jviau
Copy link
Contributor

jviau commented Oct 6, 2025

Thanks @cataggar. I am going to wait on review/sign off from @mattchenderson. He keeps an eye on Azure.Identity changes as it has been a source of many regressions for customers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

Copilot code review Copilot Copilot left review comments

@mattchenderson mattchenderson Awaiting requested review from mattchenderson

@fabiocav fabiocav Awaiting requested review from fabiocav fabiocav is a code owner

@brettsam brettsam Awaiting requested review from brettsam brettsam is a code owner

@mathewc mathewc Awaiting requested review from mathewc mathewc is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

update to Azure.Identity >= 1.15
3 participants
@cataggar @jviau @Copilot